@sunxiaojian , Can you please check all places in the project and fix them all, these two places are just examples.

@sunxiaojian , Can you please check all places in the project and fix them all, these two places are just examples.

ok, I checked. I'll double-check again.

@sunxiaojian , Can you please check all places in the project and fix them all, these two places are just examples.

ok, I checked. I'll double-check again.

At least TableEntity should be in the change list.

@sunxiaojian , Can you please check all places in the project and fix them all, these two places are just examples.

ok, I checked. I'll double-check again.

At least TableEntity should be in the change list.

fixed

@jerqi Would you like to take a look?

Member

Actually I think this is weird that null equals to empty collection.

Member

Actually I think this is weird that null equals to empty collection.

@jerqi Then let's replace it

public boolean safeIsEqualCollection(Collection<?> list1, Collection<?> list2) {
    if (list1 == list2) {
        return true;
    }
    if (list1 == null || list2 == null) {
        return false;
    }
    return CollectionUtils.isEqualCollection(list1, list2);
}

@sunxiaojian could you help to check if this issue can solve the https://github.com/apache/gravitino/pull/6455/files#r1955682884 by the way?

@sunxiaojian could you help to check if this issue can solve the https://github.com/apache/gravitino/pull/6455/files#r1955682884 by the way?

@yuqi1129 Solved it, I removed this code.

@yuqi1129 @jerqi fixed all, PTAL.

@sunxiaojian
Could you please resolve the conflict?